Skip to content

Document Session ID for Agent Observability Annotation Contexts - #38734

Closed
sabrenner wants to merge 4 commits into
masterfrom
sabrenner/session-id-annotation-context
Closed

sabrenner wants to merge 4 commits into
masterfrom
sabrenner/session-id-annotation-context

Conversation

@sabrenner

Copy link
Copy Markdown
Contributor

What does this PR do? What is the motivation?

Adds examples in Agent Observability SDK documentation for tagging session_id for auto-instrumented spans via annotation contexts.

Merge readiness

  • Ready for merge

For Datadog employees:

  • ⚠️ Your branch name MUST follow the <name>/<description> convention and include the forward slash (/). If you've already created your PR with an incorrect branch name, please rename your branch and open a fresh PR.
  • 🤖 New: Comment with /review to run an automated check that catches common issues before a Documentation team member reviews your PR.

AI assistance

Additional notes

@sabrenner
sabrenner requested a review from a team as a code owner August 4, 2026 13:36
@github-actions

github-actions Bot commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Preview links (active after the build_preview check completes)

Modified Files

@datadog-datadog-prod-us1

This comment has been minimized.

@rtrieu

rtrieu commented Aug 4, 2026

Copy link
Copy Markdown
Collaborator

/review

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 Automated review by Claude. AI-generated; verify before acting.

One functional issue in the new Node.js example (missing async keyword). Otherwise the additions are consistent with the surrounding examples.

Reviewed 6729ca4fbea9a1e1845a67ba38cc13d10af818d0 — workflow run

@rtrieu rtrieu left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hi @sabrenner, thanks for your PR! our review bot caught one issue that should be addressed. otherwise, this looks good!

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@sabrenner

Copy link
Copy Markdown
Contributor Author

@rtrieu thank you! i added the specified fix 😄 should be good for a re-review!

@jhgilbert

Copy link
Copy Markdown
Collaborator

[Automated message] This PR has merge conflicts caused by the recent docs repo reorg (files moved from the repo root into hugo/). A new PR with your commits translated to the correct paths has been opened: #38866

Please follow the instructions in the PR description.

This is an automated comment, but if you have a question, you can mention me in this PR (external contributors) or reach out in #docs-repo-reorg-support on Slack (internal contributors).

@jhgilbert jhgilbert closed this Aug 6, 2026
@jhgilbert jhgilbert added astro-reorg-autofixed Needs manual conflict resolution after replatforming reorg astro-reorg-processed Needs manual conflict resolution after replatforming reorg labels Aug 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

astro-reorg-autofixed Needs manual conflict resolution after replatforming reorg astro-reorg-processed Needs manual conflict resolution after replatforming reorg

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants